Skip to content

Validate that time property is valid and not in the future#749

Merged
stof merged 1 commit intomasterfrom
validator-validate-time-property
Apr 10, 2025
Merged

Validate that time property is valid and not in the future#749
stof merged 1 commit intomasterfrom
validator-validate-time-property

Conversation

@glaubinix
Copy link
Contributor

@glaubinix glaubinix commented Apr 10, 2025

Following #748

Validates that:

  • if time is provided as a string that it can be parsed by PHP
  • the provided time isn't in the future (which isn't possible as the time references an existing commit)

@stof stof merged commit 2f761bf into master Apr 10, 2025
2 checks passed
@stof stof deleted the validator-validate-time-property branch April 10, 2025 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@naderman naderman naderman approved these changes

@stof stof stof approved these changes

@xabbuh xabbuh xabbuh approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@glaubinix @naderman @stof @xabbuh